ShellAppMonitor: Fix case of not seeing apps being closed

Remove the app from the window_to_app hash *before* emitting
CHANGED, otherwise a signal handler could see stale data.

diff --git a/src/shell-app-monitor.c b/src/shell-app-monitor.c
index 57107047eae64e476372e7c37ff62487d743cfdc..f3bf7050ef596a0113e21c0bea41f3cb5bc8bb4d 100644 (file)
--- a/src/shell-app-monitor.c
+++ b/src/shell-app-monitor.c
@@ -532,10 +532,11 @@ shell_app_monitor_on_window_removed (MetaWorkspace   *workspace,
     self->watched_window = NULL;
 
   usage->window_count--;
+  /* Remove before emitting */
+  g_hash_table_remove (self->window_to_app, window);
+
   if (usage->window_count == 0)
     g_signal_emit (self, signals[CHANGED], 0);
-
-  g_hash_table_remove (self->window_to_app, window);
 }
 
 static void